Skip to content

Doc securing services : the bundle JMS no longer recommend for Sf4 #9398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 7, 2018

Conversation

Ismailmezrani
Copy link
Contributor

Until now the JMSSecurityExtraBundle not support Symfony 4.*,
=> So we can't install it.

Until now the JMSSecurityExtraBundle not support Symfony 4.*, 
=> So we can't install it.
@Ismailmezrani Ismailmezrani changed the title the bundle no longer recommend for Sf4 Doc securing_services : the bundle JMS no longer recommend for Sf4 Mar 6, 2018
@Ismailmezrani Ismailmezrani changed the title Doc securing_services : the bundle JMS no longer recommend for Sf4 Doc securing services : the bundle JMS no longer recommend for Sf4 Mar 6, 2018
@javiereguiluz
Copy link
Member

Let's merge this because some JMS bundles are in a bad state and this recommendation is also controversial. Not sure if we want to promote putting annotations in service methods to enable security. @Ismailmezrani thanks and congrats on your first Symfony Docs contribution.

@javiereguiluz javiereguiluz merged commit 57d08de into symfony:4.0 Mar 7, 2018
javiereguiluz added a commit that referenced this pull request Mar 7, 2018
…d for Sf4 (Ismailmezrani)

This PR was merged into the 4.0 branch.

Discussion
----------

Doc securing services : the bundle JMS no longer recommend for Sf4

Until now the JMSSecurityExtraBundle not support Symfony 4.*,
=> So we can't install it.

Commits
-------

57d08de the bundle no longer recommend for Sf4
@Ismailmezrani Ismailmezrani deleted the patch-1 branch March 7, 2018 09:00
@xabbuh xabbuh added this to the 4.0 milestone Jul 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants